-
-
Notifications
You must be signed in to change notification settings - Fork 899
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
docs: add note about cypto.randomUUID()
#686
Conversation
@@ -32,7 +32,11 @@ For the creation of [RFC4122](https://www.ietf.org/rfc/rfc4122.txt) UUIDs | |||
- **Small** - Zero-dependency, small footprint, plays nice with "tree shaking" packagers | |||
- **CLI** - Includes the [`uuid` command line](#command-line) utility | |||
|
|||
**Upgrading from `uuid@3`?** Your code is probably okay, but check out [Upgrading From `uuid@3`](#upgrading-from-uuid3) for details. | |||
> **Note** |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This applies some special GitHub flavored markdown to visualize the notes:
Note
Upgrading fromuuid@3
? Your code is probably okay, but check out Upgrading Fromuuid@3
for details.
Note
Only interested in creating a version 4 UUID? You might be able to usecypto.randomUUID()
, eliminating the need to install this library.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool! I didn't know about this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes looks good 👍
Could you run npm run docs
so that the rendered readme is also updated?
ba449b6
to
711f1b6
Compare
cypto.randomUUID()
cypto.randomUUID()
@LinusU Done 👍 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
@jonkoops Thanks for the help! |
Closes #682